Skip to content

luci-mod-network: mptcp option uses '' instead of "off"#8586

Merged
systemcrash merged 1 commit into
openwrt:masterfrom
AndyChiang888:mptcp
May 16, 2026
Merged

luci-mod-network: mptcp option uses '' instead of "off"#8586
systemcrash merged 1 commit into
openwrt:masterfrom
AndyChiang888:mptcp

Conversation

@AndyChiang888
Copy link
Copy Markdown
Contributor

In the mptcp options, "off" is the default option, which has the same effect as not defining it. Using '' will prevent unnecessary parameters from being written to the flash.

Link https://openwrt.org/docs/guide-user/network/mptcp

Pull request details

Description

Screenshot or video of changes (if applicable)

Maintainer (preferred)

@


Tested on

OpenWrt version: SNAPSHOT
LuCI version: master
Web browser(s): Chrome 147.0.7727.102


Checklist

  • This PR is not from my main or master branch 💩, but a separate branch. ✅
  • Each commit has a valid ✒️ Signed-off-by: <my@email.address> row (via git commit --signoff).
  • Each commit and PR title has a valid 📝 <package name>: title first line subject for packages.
  • Incremented 🆙 any PKG_VERSION in the Makefile.
  • (Optional) Includes what Issue it closes (e.g. openwrt/luci#issue-number).
  • (Optional) Includes what it depends on (e.g. openwrt/packages#pr-number in sister repo).

@github-actions

This comment has been minimized.

@systemcrash
Copy link
Copy Markdown
Contributor

@dangowrt can you confirm what the values should be? The wiki says otherwise.

https://openwrt.org/docs/guide-user/network/mptcp

Comment thread modules/luci-mod-network/htdocs/luci-static/resources/view/network/interfaces.js Outdated
In the mptcp options, "off" is the default option, which has the same effect as not defining it.
Using '' will prevent unnecessary parameters from being written to the flash.

Link https://openwrt.org/docs/guide-user/network/mptcp

Signed-off-by: Andy Chiang <AndyChiang_git@outlook.com>
@github-actions
Copy link
Copy Markdown

Warning

Some formality checks failed.

Consider (re)reading submissions guidelines.

Failed checks

Issues marked with an ❌ are failing checks.

Commit ffd48fa

  • 🔶 Commit message lines should be <= 75 characters long
    Actual: line 2 is 96 characters long
    $\textsf{In the mptcp options, "off" is the default option, which has the same effec\color{red}{t as not defining it.}}$
    Actual: line 3 is 77 characters long
    $\textsf{Using '' will prevent unnecessary parameters from being written to the flas\color{red}{h.}}$

For more details, see the full job log.

Something broken? Consider providing feedback.

@systemcrash systemcrash merged commit 317ff9d into openwrt:master May 16, 2026
5 of 6 checks passed
@AndyChiang888 AndyChiang888 deleted the mptcp branch May 16, 2026 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants